Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(helm): allow setting affinity for deployment #686

Merged
merged 1 commit into from
Jul 8, 2024
Merged

Conversation

apricote
Copy link
Member

@apricote apricote commented Jul 8, 2024

Support setting the Affinity for pods of the Deployment.

@apricote apricote added the enhancement New feature or request label Jul 8, 2024
@apricote apricote self-assigned this Jul 8, 2024
@apricote apricote requested a review from a team as a code owner July 8, 2024 08:55
Copy link
Member

@jooola jooola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume you tested it? No need for a snapshot?

@apricote
Copy link
Member Author

apricote commented Jul 8, 2024

Yup, tested it locally. Its the exact same code as for the nodeSelector right above it. Also tested by #687.

We do not have helm snapshots tests for this repo yet.

Copy link

codecov bot commented Jul 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.16%. Comparing base (672f8e1) to head (cc6fac8).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #686      +/-   ##
==========================================
- Coverage   72.26%   72.16%   -0.10%     
==========================================
  Files          32       31       -1     
  Lines        2668     2497     -171     
==========================================
- Hits         1928     1802     -126     
+ Misses        552      523      -29     
+ Partials      188      172      -16     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@apricote
Copy link
Member Author

apricote commented Jul 8, 2024

Added an issue for the snapshots: #688

@apricote
Copy link
Member Author

apricote commented Jul 8, 2024

CI failures are expected due to #687.

@apricote apricote merged commit 1a8ea95 into main Jul 8, 2024
5 of 8 checks passed
@apricote apricote deleted the helm-affinity branch July 8, 2024 09:02
apricote pushed a commit that referenced this pull request Jul 8, 2024
🤖 I have created a release *beep* *boop*
---


##
[1.20.0](v1.19.0...v1.20.0)
(2024-07-08)


### Features

* add support & tests for Kubernetes 1.29
([#600](#600))
([e8fabda](e8fabda))
* add support & tests for Kubernetes 1.30
([#679](#679))
([0748b6e](0748b6e))
* drop tests for kubernetes v1.25
([#597](#597))
([58261ec](58261ec))
* drop tests for kubernetes v1.26
([#680](#680))
([9c4be01](9c4be01))
* emit Kubernetes events for error conditions
([#598](#598))
([e8f9199](e8f9199))
* **helm,manifests:** only specify container args instead of command
([#691](#691))
([2ba4058](2ba4058))
* **helm:** allow setting affinity for deployment
([#686](#686))
([1a8ea95](1a8ea95))
* read HCLOUD_TOKEN from file
([#652](#652))
([a4343b8](a4343b8))


### Bug Fixes

* **routes:** many requests for outdated routes by rate limiting
([#675](#675))
([e283b7d](e283b7d))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants